Skip to content

Update usage of shared stream buffer #1016

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
praasz merged 7 commits into from
Nov 13, 2025
Merged

Update usage of shared stream buffer #1016

praasz merged 7 commits into from
Nov 13, 2025
+22 −11

Conversation

@praasz
Copy link
Contributor

@praasz praasz commented Oct 29, 2025

Description

  • Updates to compatible with changes in ov::Tensor::data() member

@praasz praasz requested a review from a team as a code owner October 29, 2025 15:15
@github-actions github-actions bot added the category: NVIDIA plugin OpenVINO NVIDIA plugin label Oct 29, 2025
@praasz praasz requested a review from a team as a code owner October 30, 2025 07:31
@github-actions github-actions bot added dependencies Pull requests that update a dependency file category: custom operations OpenVINO Runtime Extension with custom operations labels Oct 30, 2025
@praasz praasz mentioned this pull request Oct 30, 2025
Merged
@praasz
Copy link
Contributor Author

praasz commented Oct 30, 2025

@apaniukov
Could you check the failing python test?
The issue could because of same python pkg is still missing or not compatible or these are expected failures or something must be updated?
The PR doesn't change custom op module code.

@apaniukov
Copy link
Contributor

apaniukov commented Oct 30, 2025

The error comes from OnnxFrontEnd.
OV tests use version <= 1.17.0, while collected version is 1.19.1: https://github.com/openvinotoolkit/openvino_contrib/actions/runs/18933221626/job/54057400848?pr=1016#step:21:144
@dorloff what is the current supported onnx version?

@apaniukov apaniukov requested a review from dorloff October 30, 2025 12:37
@praasz praasz requested a review from apaniukov November 3, 2025 13:58
@praasz praasz enabled auto-merge (squash) November 5, 2025 14:35
@praasz
Copy link
Contributor Author

praasz commented Nov 6, 2025

@apaniukov, @dorloff Could you look again at this PR, I've updated extension but it looks model generation is not compatible what was created some time ago.
I've also disabled not passing test but still extension works but it looks lie custom operators should be updated to give correct result

@praasz praasz merged commit ad5bb54 into openvinotoolkit:master Nov 13, 2025
20 checks passed
github-merge-queue bot pushed a commit to openvinotoolkit/openvino that referenced this pull request Nov 13, 2025
@praasz @mlukasze
[core] Remove and change deprecated data() member in tensor (#32569)
fa61863 
### Details:
- Remove deprecate notice and `ov::Tensor::data()` returns const pointer
for const Tensor
 
 ### Blocked by:
 - openvinotoolkit/openvino_contrib#1016
 - openvinotoolkit/openvino.genai#2934
 - openvinotoolkit/openvino.genai#2978
 - openvinotoolkit/openvino.genai#3009

### Tickets:
 - CVS-174872

---------

Signed-off-by: Raasz, Pawel <[email protected]>
Co-authored-by: Michal Lukaszewski <[email protected]>
@praasz praasz deleted the deprecate/update-for-ov-tensor-api-change branch November 19, 2025 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@apaniukov apaniukov apaniukov approved these changes

@p-durandin p-durandin p-durandin approved these changes

@dorloff dorloff Awaiting requested review from dorloff

Assignees

@apaniukov apaniukov

Labels

category: custom operations OpenVINO Runtime Extension with custom operations category: NVIDIA plugin OpenVINO NVIDIA plugin dependencies Pull requests that update a dependency file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@praasz @apaniukov @p-durandin